Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI] Disable test_ios_e2e and test_js_e2e tests #25248

Closed
wants to merge 1 commit into from

Conversation

hramos
Copy link
Contributor

@hramos hramos commented Jun 13, 2019

Disables the test_ios_e2e and test_js_e2e tests temporarily until the regression introduced in #25190 is fixed. A fix has been identified and proposed at jestjs/jest#8558.

We can re-enable these once a release of Jest with a patched jest-haste-map is available.

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner RN Team labels Jun 13, 2019
@hramos hramos requested a review from cpojer June 13, 2019 05:16
@react-native-bot react-native-bot added the Platform: iOS iOS applications. label Jun 13, 2019
@cpojer
Copy link
Contributor

cpojer commented Jun 13, 2019

The Jest PR got merged, I'll land the CLI upgrade that will fix CI.

@cpojer cpojer closed this Jun 13, 2019
@cpojer
Copy link
Contributor

cpojer commented Jun 13, 2019

Even with the latest CLI upgrade, we are now running into issues on CI about the amount of files we are opening: #25241 (comment)

We are trying to resolve it but if we can't today, please feel free to reopen and merge this PR.

@cpojer
Copy link
Contributor

cpojer commented Jun 13, 2019

Landing #25257 which should fix CI. Thank you @thymikee.

@hramos
Copy link
Contributor Author

hramos commented Jun 13, 2019

Thanks @thymikee and @cpojer!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Facebook Partner: Facebook Partner Platform: iOS iOS applications. RN Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants